Skip to content

Conversation

@HardNorth
Copy link
Member

No description provided.

@HardNorth HardNorth requested a review from Copilot March 28, 2025 10:28
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR prepares a release by bumping the package version and introducing a debug mode feature. Key changes include:

  • Updating the version in setup.py from 5.6.3 to 5.6.4.
  • Adding a new "debug_mode" attribute to support detailed error logging in variables.py and propagating it in service.py.
  • Documenting the RP_DEBUG_MODE configuration in the changelog.

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
setup.py Bumped version to 5.6.4.
robotframework_reportportal/variables.py Added a debug_mode field and initialized it from RP_DEBUG_MODE.
robotframework_reportportal/service.py Wrapped several methods in try/except blocks to log errors when debug is enabled.
CHANGELOG.md Updated to document the new RP_DEBUG_MODE configuration.
Comments suppressed due to low confidence (2)

robotframework_reportportal/service.py:135

  • The new try/except block added for start_launch uses debug mode to log errors; please add tests to verify that error logging behaves correctly when debug mode is enabled.
try:

robotframework_reportportal/variables.py:139

  • The initialization of debug_mode from RP_DEBUG_MODE is a new behavior; please add tests to confirm its behavior across different RP_DEBUG_MODE settings.
self.debug_mode = to_bool(get_variable("RP_DEBUG_MODE", default="False"))

@HardNorth HardNorth merged commit 984973e into master Mar 28, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant